-
Notifications
You must be signed in to change notification settings - Fork 232
Build: Bump Qt6 from 6.8.1 to 6.9.1 (Automated PR) #3407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
cf2dc7c
to
e54a6c6
Compare
e54a6c6
to
4209297
Compare
4fef397
to
908b7f4
Compare
908b7f4
to
7d6c516
Compare
7d6c516
to
337d5e6
Compare
42fba64
to
3f643c1
Compare
3f643c1
to
60eeada
Compare
PR has been updated for version 6.8.1 by the workflow Bump dependencies (create-prs). |
3262e16
to
e3d19a9
Compare
a2b5b6c
to
42eccc7
Compare
I'd assume that this breaks on every Qt6 build. |
OK, this could have an impact on our plans for Jamulus 4 but it shouldn't hold up 3.12.0, I agree. |
The linked issue from Qt has had an update. It's now in progress |
0e95992
to
360c79e
Compare
360c79e
to
148611a
Compare
PR has been updated for version 6.8.3 by the workflow Bump dependencies (create-prs). |
148611a
to
0207836
Compare
0207836
to
d1220d0
Compare
PR has been updated for version 6.9.0 by the workflow Bump dependencies (create-prs). |
OK, so now that's even noisier about FFMpeg dependencies not met. Have they made it clearer how to remove the dependency? |
I believe that the next version should have a fix. |
2e27fba
to
4bb1767
Compare
4bb1767
to
d376fe5
Compare
aee5f0b
to
98c9a86
Compare
PR has been updated for version 6.9.1 by the workflow Bump dependencies (create-prs). |
98c9a86
to
9a92af9
Compare
Still failing -- are we turning on stuff that's not needed for audio?
|
Maybe that's needed. I thought they fixed it on the Qt side... |
Ah: https://codereview.qt-project.org/c/qt/qtmultimedia/+/630015 We might need to add |
OK, having read up a bit I'm convinced we stick with the Qt-preferred approach and use FFmpeg... Even if we did
that wouldn't remove the dependency, just tell the runtime to ignore FFmpeg and use darwin directly. So yeah, https://doc.qt.io/qt-6/qtmultimedia-apple.html:
looks like it should do the trick -- although it sounds like that's also a deployment time issue:
I'm not sure how much it'll help at build time. |
CoPilot took some convincing that it was wrong about this as usual (now out of date documentation being more prevalently referenced in once-correct answers). It's suggested we'd need to amend autobuild.yml (and ios.sh potentially) to build/cache the ffmpeg libraries for the link stage to work. |
I also asked if there were any existing actions - but none for iOS, according to CoPilot. It's top answer was:
https://github.com/copilot/share/42365326-43c0-8404-9102-2648a4486807 Now, given that the first of those is FFmpeg 4 and the current version is 7 and the second of those has been archived... I'm thinking we might want to brew from https://github.com/FFmpeg/FFmpeg -- but it'd need someone to knock up the build, possible from cobbling it up from the above. (It does seem to be becoming increasingly difficult to support Qt6 as time progresses...) https://doc.qt.io/qt-6/qtmultimedia-building-ffmpeg-ios.html -- there's actually an official page on how hard it is... |
This automated Pull Request updates the used Qt6 version to version 6.9.1.
This PR was opened by the workflow Bump dependencies (create-prs)
CHANGELOG: Build: Updated bundled Qt6 to version 6.9.1